Test results for "tests 1"

37247 passed, 650 skipped
✔️✔️✔️

Merge workflow run.

Would you consider reverting this change?

We have a script that mirrors the Playwright artifacts into our own internal repo (so we don't put load on and rely on the Azure CDNs in our CI). We use browsers.json to know which artifacts/versions are required by the latest version of Playwright. By moving this out of browsers.json, browsers.json is no longer a complete source of "what artifacts/versions does Playwright require?"

I certainly could update my script, but I'd prefer not to have to port all the logic that may be introduced here just to know which artifacts to download. Given that browsers.json is generated, it seems better to keep this logic in roll_browser.js so that browsers.json is an authoritative source of this information.

I've put up a PR in #34636.

If there is a better way for us to collect the list of artifacts/versions Playwright uses, I'd be happy to update our script.

@pkoenig10, browsers.json is not part of our public API and we do not condone this use. See a similar issue from last week: #34508 (comment)

We have ended up reverting the change for browsers.json. We still do not condone relying on this file, but until we provide a more suitable API, we will unblock you. This will release with v1.51.